Skip to content

fix: render task list markers correctly with Comark#2576

Open
cyphercodes wants to merge 1 commit into
slidevjs:mainfrom
cyphercodes:fix/task-list-comark-marker
Open

fix: render task list markers correctly with Comark#2576
cyphercodes wants to merge 1 commit into
slidevjs:mainfrom
cyphercodes:fix/task-list-comark-marker

Conversation

@cyphercodes
Copy link
Copy Markdown
Contributor

Summary

  • Wrap Slidev's task list plugin setup so Comark/MDC inline spans generated from [x] / [ ] markers are removed after checkboxes are created.
  • Add regression coverage for checked and unchecked task list items rendered with Comark enabled.

Fixes #2568

Test plan

  • pnpm exec vitest run packages/slidev/node/syntax/task-list.test.ts
  • pnpm exec vitest run packages/slidev/node/syntax
  • pnpm exec eslint packages/slidev/node/syntax/index.ts packages/slidev/node/syntax/task-list.ts packages/slidev/node/syntax/task-list.test.ts
  • pnpm --filter @slidev/cli build
  • git diff --check

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 30, 2026

Deploy Preview for slidev ready!

Name Link
🔨 Latest commit 566f838
🔍 Latest deploy log https://app.netlify.com/projects/slidev/deploys/69f3e921225c5d0008f60c96
😎 Deploy Preview https://deploy-preview-2576--slidev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 2, 2026

Open in StackBlitz

@slidev/client

npm i https://pkg.pr.new/@slidev/client@2576

create-slidev

npm i https://pkg.pr.new/create-slidev@2576

create-slidev-theme

npm i https://pkg.pr.new/create-slidev-theme@2576

@slidev/parser

npm i https://pkg.pr.new/@slidev/parser@2576

@slidev/cli

npm i https://pkg.pr.new/@slidev/cli@2576

@slidev/types

npm i https://pkg.pr.new/@slidev/types@2576

commit: 566f838

@kermanx
Copy link
Copy Markdown
Member

kermanx commented May 2, 2026

Thank you for the contribution. I think it may be better to propose a fix on the Comark side, like comarkdown/markdown-it-comark#29.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Markdown Lists with Ceckboxes Not Displayed Properly

2 participants